Skip to content

fix: Correctly match version placeholder in build workflow - #189

Merged
Stephan Wenger (swenger) merged 9 commits into
mainfrom
swenger-fix-sed
Aug 26, 2026
Merged

fix: Correctly match version placeholder in build workflow#189
Stephan Wenger (swenger) merged 9 commits into
mainfrom
swenger-fix-sed

Conversation

@swenger

@swenger Stephan Wenger (swenger) commented Aug 25, 2026

Copy link
Copy Markdown
Contributor

Motivation

Bug in container build workflow: Version constraints that legitimately contain 0.0.0 (like >=0.0.0) were broken by the sed command. The special semantics of . as a match-all character make it likely that any version containing three zeros separated by some other character would break.

Co-authored-by: swenger <603549+swenger@users.noreply.github.com>
Copilot AI and others added 2 commits August 25, 2026 10:33
Co-authored-by: swenger <603549+swenger@users.noreply.github.com>
Co-authored-by: swenger <603549+swenger@users.noreply.github.com>
Co-authored-by: swenger <603549+swenger@users.noreply.github.com>
Comment thread tests/test_template.py Outdated
Co-authored-by: swenger <603549+swenger@users.noreply.github.com>
Comment thread template/pixi.toml.jinja Outdated
Comment thread tests/test_template.py Outdated
Co-authored-by: Stephan Wenger <swenger@users.noreply.github.com>
@swenger
Stephan Wenger (swenger) merged commit 0427ff8 into main Aug 26, 2026
6 checks passed
@swenger
Stephan Wenger (swenger) deleted the swenger-fix-sed branch August 26, 2026 07:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants